Auto merge of #3426 - nrc:check-fix-2, r=alexcrichton

Test for #3419

Based on top of https://github.com/rust-lang/cargo/pull/3425.

I'm not sure if this is an acceptable test because it downloads rustc-serialize.

r? @alexcrichton

Auto merge of #3436 - matklad:abs-target, r=alexcrichton

Make sure targets always use absolute path

Closes #3416

I'd want to add a test for it later. Seems a bit suspicious that this change has not caused any test to fail.

Auto merge of #3457 - alexcrichton:no-check-on-beta, r=alexcrichton

Only run check tests on nightly

Only run check tests on nightly

Test for cargo check with macro and both lib and bin.

Closes #3419

Add a test

Includes adding facility for checking that output does not contain a string.

Make sure targets always use absolute path

Auto merge of #3430 - alexcrichton:bump, r=brson

Bump to 0.17.0

Bump to 0.17.0

Auto merge of #3424 - matklad:naming-things, r=alexcrichton

Minor, more explicit names

Ultraminor renaming. Have to look at this tests because it fails if you link with older libgit.

Auto merge of #3413 - matklad:ref, r=alexcrichton

Refactor resolve_dependencies

This moves `resolve_dependencies` function from `compile.rs` into the `resolve.rs`, where it more reasonably belongs.

Auto merge of #3410 - fmdkdd:fix-3390-message-format-parse-stderr, r=alexcrichton

Fix `--message-format JSON` when rustc emits non-JSON warnings

The `--message-format JSON` flag parses all the stderr output of rustc to JSON,
but rustc can emit non-JSON lines to stderr (e.g., for warning about the
unstable `-Z` flag on the stable channel), causing cargo to fail reporting
compilation errors when using `--message-format JSON`.

This commit adds a check to look for lines beginning with `{` to only parse
these lines as JSON.  Other lines from rustc are forwarded to the stderr of
cargo.

Fixes #3390.

make sure we use the check profile where possible for lib deps.

Fixes #3418

Minor, more explicit names

Don't carry source ids along with sources

Be more explicit about temporary workspaces

Add source_id method to the Source

Add a test for `--message-format` parsing the mixed output of rustc

Rename resolve_dependencies -> resolve_ws_precisely

Simplify `resolve_ws` API

Move `resolve_dependencies` into resolve.rs

Auto merge of #3411 - alexcrichton:tweaks, r=alexcrichton

Minor build tweaks

* Clean out OpenSSL when building
* Test 32-bit Cargo on OSX, not the 64-bit version on the 32-bit builder

Minor build tweaks

* Clean out OpenSSL when building
* Test 32-bit Cargo on OSX, not the 64-bit version on the 32-bit builder

Fix `--message-format JSON` when rustc emits non-JSON warnings

The `--message-format JSON` flag parses all the stderr output of rustc to JSON,
but rustc can emit non-JSON lines to stderr (e.g., for warning about the
unstable `-Z` flag on the stable channel), causing cargo to fail reporting
compilation errors when using `--message-format JSON`.

This commit adds a check to look for lines beginning with `{` to only parse
these lines as JSON.  Other lines from rustc are forwarded to the stderr of
cargo.

Fixes #3390.

Auto merge of #3409 - matklad:workspace-manifest-error, r=alexcrichton

Don't ignore errors in workspace manifest

Closes #3407

If we find `Cargo.toml` during an up tree traversal, we'd better not to ignore errors within it!

Don't ignore errors in workspace manifest

Auto merge of #3393 - alexcrichton:fix-libz-linkage, r=brson

Bump libz-sys to 1.0.9

This just switches libz to always link statically instead of relying on the
system zlib. For MinGW it seems that linkage may default pull in a DLL, which is
almost never what we want.

Closes #3384

Auto merge of #3403 - matklad:doc-loc, r=alexcrichton

cargo doc: show where are docs

Let's print the file we are trying to open. It's useful when we fail to detect the browser, because the user would be able to copy-paste url manually.

Bump libz-sys to 1.0.9

This just switches libz to always link statically instead of relying on the
system zlib. For MinGW it seems that linkage may default pull in a DLL, which is
almost never what we want.

Also update curl-sys to fix a build issue on MinGW.

Closes #3384

Auto merge of #3402 - matklad:packages, r=alexcrichton

Refactor packages specification

A small refactoring to avoid returning a triple.

cargo doc: show where are docs

Refactor packages specification

Auto merge of #3396 - pornel:test, r=alexcrichton

Test for issue #2064

I've tried to reproduce the problem from #2064, but to my surprise the test passes reliably :)

Auto merge of #3296 - nrc:check, r=alexcrichton

cargo check

~~This is not finished - the big omission is no tests, and it needs some more testing too. It also requires https://github.com/rust-lang/rust/pull/37681.~~

However, this is my first non-trivial Cargo patch, so I'd like to get some early feedback.

r? @alexcrichton

and cc @rust-lang/tools since this adds a new Cargo sub-command (although we have previously discussed and approved the idea in a tools meeting).

Auto merge of #3398 - alexcrichton:git-rev, r=alexcrichton

Fix git rev showing up in version number

Makefiles didn't support an out-of-tree build, so needed to update them to do
so.

Auto merge of #3363 - alexcrichton:libcmt, r=alexcrichton

Compile statically against the MSVC CRT

This updates our AppVeyor builds to compile with `-Ctarget-feature=+crt-static`
to help Cargo be a bit more portable and not rely on the MSVC redistributable
artifacts. Over time this may even let us converge on only releasing one build
of Cargo and just pairing that with all Windows toolchains...

Auto merge of #3399 - alexcrichton:fix-nightly, r=alexcrichton

Fix compatibility with Rust nightly

These two tests actually shouldn't have ever passed, but nightly Rust is more
principled about "$OUT_DIR" and doesn't leak it where possible, so these two
tests were accidentally compiling due to leaking '$OUT_DIR' for Cargo itself.

Auto merge of #3386 - matklad:warn-replace, r=alexcrichton

Warn if replace is not actually used

closes #3324

I've added warnings to `resolve_dependencies` inside `cargo_compile.rs`. Adding them to `resolve_with_previous` does not work because that method is called several times.

Perhaps the best idea is to refactor resolve facade, such that we have only `ops::resolve`  and not `ops::resolve_ws`, `ops::resolve_with_previous` and `ops::resolve_dependencies`. IIRC, there were some bugs because of logic mismatch between `compile` and `metadata` commands.

Fix compatibility with Rust nightly

These two tests actually shouldn't have ever passed, but nightly Rust is more
principled about "$OUT_DIR" and doesn't leak it where possible, so these two
tests were accidentally compiling due to leaking '$OUT_DIR' for Cargo itself.

Fix git rev showing up in version number

Makefiles didn't support an out-of-tree build, so needed to update them to do
so.

Auto merge of #3385 - matklad:package-test, r=alexcrichton

Add test for --package and virtual manifest

closes #3194

The issue was actually fixed by #3221 (thanks @euclio !), so let's just add a test (a copy of `virtual_works` basically).

Auto merge of #3371 - trafficland:master, r=alexcrichton

Add -vv to the list of options printed by cargo --help.

Compile statically against the MSVC CRT

This updates our AppVeyor builds to compile with `-Ctarget-feature=+crt-static`
to help Cargo be a bit more portable and not rely on the MSVC redistributable
artifacts. Over time this may even let us converge on only releasing one build
of Cargo and just pairing that with all Windows toolchains...

Test for issue #2064

Auto merge of #3395 - alexcrichton:fix-appveyor, r=alexcrichton

Pin to an older version of rustup temporarily

cc #3394

Pin to an older version of rustup temporarily

cc #3394

Fix some rebasing errors

Auto merge of #3392 - matklad:patch-1, r=steveklabnik

Add explanatory comment to header.html

See #3387

Add explanatory comment to header.html

See #3387

Warn if replace is not actually used

closes #3324

Add test for --package and virtual manifest

closes #3194

Added -vv to the list of options printed by cargo and its sub commands.

Be a bit more careful not to crash if the compiler doesn't support --crate-type metadata

tests

cargo check

Adds a new mode - check - which only checks code, rather than generating machine code. It takes into account that proc macros and build scripts will still require generated code.

Implemented by adding a check profile and setting this on each Unit, unless the unit is required to be built (i.e., is a proc macro, build script, or dep of one).

Auto merge of #3221 - euclio:test-all, r=alexcrichton

Add `--all` flag to `cargo test`

Work towards #2878.

This flag allows a user to test all members of a workspace by using `cargo test --all`. The command is also supported when using a virtual workspace manifest.

replace `current_opt` with membership check

fix formatting issues

remove unused argument to `Context::lib_profile`

remove unnecessary usage of `current_opt`

avoid round-tripping through string

remove unnecessary match

show warnings for all members of the workspace

pass `Packages` into resolve dependencies

remove current_package from context

add `--all` flag to `cargo-test`

avoid panic when virtual workspace has no members

Auto merge of #3361 - pwoolcoc:default-build-script, r=alexcrichton

Assume `build.rs` in the same directory as `Cargo.toml` is a build script (unless explicitly told not to)

So, in May I posted a question in the #cargo IRC room: https://botbot.me/mozilla/cargo/2016-05-26/?msg=66770068&page=1

This PR does what was discussed there. If `cargo` sees a `build.rs` in the same directory as the `Cargo.toml`, it will assume `build.rs` is a build script unless `package.build = false`. Just for completeness I also made `build = true` mean the same as `build = "build.rs"` but I'm not sure if that is okay or not.

change this to only emit a warning right now

Auto merge of #3372 - alexcrichton:sha256, r=alexcrichton

Upload sha256 sums of cargo artifacts

Upload sha256 sums of cargo artifacts

Auto merge of #3319 - alexcrichton:more-info, r=brson

Emit more info on --message-format=json

This adds more output on Cargo's behalf to the output of
`--message-format=json`. Cargo will now emit a message when a crate is finished
compiling with a list of all files that were just generated along with a message
when a build script finishes executing with linked libraries and linked library
paths.

Closes #3212

correct typo

Using layout.root here should be sufficient

Added .idea to the list of git exlusions.

Auto merge of #3356 - ibabushkin:feature-build-rustflags, r=alexcrichton

Implemented string lookup for `build.rustflags` config key

This addresses the immediate issue described in #3052 .
I am, however, unsure about the current state of the deeper issues mentioned in that issue, but if needed, I can take stab at them as well. :)

Auto merge of #3362 - alexcrichton:slower-ssl, r=alexcrichton

Use -j1 on recursive make for OpenSSL

It looks like OpenSSL building on OSX has races...

Use -j1 on recursive make for OpenSSL

It looks like OpenSSL building on OSX has races...

Tests for new `package.build` behavior

Assume build = 'build.rs' by default

This change makes cargo assume `build = "build.rs"` if there is a
`build.rs` file in the same directory as `Cargo.toml`. However, you
can set `build = false` to prevent this.

Refactored ugly rustflags lookup code into a separate function.

Emit more info on --message-format=json

This adds more output on Cargo's behalf to the output of
`--message-format=json`. Cargo will now emit a message when a crate is finished
compiling with a list of all files that were just generated along with a message
when a build script finishes executing with linked libraries and linked library
paths.

Closes #3212

Auto merge of #3358 - tshepang:formatting, r=alexcrichton

doc: fix code formatting

Fixed broken testcase for string-based form of rustflags config key.

Auto merge of #3348 - alexcrichton:fix-checksums, r=brson

Fix retrying crate downloads for network errors

Previously the `with_retry` loop was a little too tight where stale state about
the sha256 and data was kept out of the loop. Instead we need to reinitialize
these on each iteration of the loop to ensure that we correctly retry by
forgetting the data we previously downloaded for an aborted download attempt.

Auto merge of #3338 - alexcrichton:correct-vers-flag, r=brson

Require `cargo install --vers` takes a semver version

Historically Cargo accidentally took a semver version *requirement*, so let's
start issuing warnings about how this is now legacy behavior.

Closes #3321

Auto merge of #3335 - alexcrichton:fix-xcomiple, r=brson

Add host dependency path via -L for cross compiles

Now that proc-macro crates can reexport from their dependencies we need to be
able to find the other crates, so ensure that we pass an appropriate -L flag.

Closes #3334

Add host dependency path via -L for cross compiles

Now that proc-macro crates can reexport from their dependencies we need to be
able to find the other crates, so ensure that we pass an appropriate -L flag.

Closes #3334

Auto merge of #3336 - alexcrichton:fix-warning, r=brson

Test for bad path overrides with summaries

Bad path overrides are currently detected to issue warnings in cases where path
overrides are not suitable and have exhibited buggy behavior in the past.
Unfortunately though it looks like some false positives are being issued,
causing unnecessary confusion about `paths` overrides.

This commit fixes the detection of these "bad path overrides" by comparing
`Summary` dependencies (what's written down in `Cargo.toml`) rather than
comparing the `Cargo.toml` of the override with `Cargo.lock`. We're guaranteed
that the package we're overridding has already been resolved into `Cargo.lock`,
so we know that if the two `Cargo.toml` files are equivalent we'll continue
with the same crate graph.

I'm not actually entirely sure why I originally thought it'd be better to go
through the `Cargo.lock` comparison route. Unfortunately that doesn't take into
account optional deps which aren't in `Cargo.lock` but are in `Cargo.toml` of
the override, causing the false positive. This method, however, simply ensures
that the two dependency lists are the same.

Closes #3313

doc: fix code formatting

Auto merge of #3310 - alexcrichton:more-metadata-hashing, r=brson

Apply new fingerprinting to build dir outputs

We now much more aggressively cache the output of the compiler based on feature
sets and profile configuration. Unfortunately we forgot to extend this caching
to build script output directories as well so this commit ensures that build
script outputs are cached the same way with a directory per configuration of
features and output settings.

Closes #3302

Improved error handling to reflect the actual situation, added tests

* One of the tests still doesn't pass, this needs further investigation

Auto merge of #3357 - tshepang:no-exist, r=steveklabnik

doc: remove reference to non-existent function

Auto merge of #3354 - alexcrichton:less-snapshots, r=alexcrichton

Delete old snapshot infrastructure

No longer used

doc: remove reference to non-existent function

Auto merge of #3355 - alexcrichton:change-buckets, r=alexcrichton

Upload to a different bucket

Implemented string lookup for `build.rustflags` config key

Upload to a different bucket

Delete old snapshot infrastructure

No longer used

Auto merge of #3350 - alexcrichton:more-tweaks, r=alexcrichton

Slight tweaks to CI

* Pass `--quiet` to all tests to have some quieter output
* Skip builds on the `master` branch as `auto` is already checked
* Check the right env var for repo branches on appveyor